Skip to content

feat: display hidden iconset#384

Merged
antfu merged 2 commits intoantfu-collective:mainfrom
zyyv:for-hidden-collections
Dec 15, 2025
Merged

feat: display hidden iconset#384
antfu merged 2 commits intoantfu-collective:mainfrom
zyyv:for-hidden-collections

Conversation

@zyyv
Copy link
Contributor

@zyyv zyyv commented Dec 3, 2025

Description

Manually manage hidden icon sets instead of excluding them outright, even if they have been marked as deprecated or unavailable.

For deprecated iconset ui
image
image

Linked Issues

Related to b74141e

Additional context

Copilot AI review requested due to automatic review settings December 3, 2025 07:40
@netlify
Copy link

netlify bot commented Dec 3, 2025

Deploy Preview for icones ready!

Name Link
🔨 Latest commit 2c75eae
🔍 Latest deploy log https://app.netlify.com/projects/icones/deploys/692ff1ef5fce56000866a46a
😎 Deploy Preview https://deploy-preview-384--icones.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR enables the display of deprecated/unavailable icon sets that were previously filtered out entirely. Instead of excluding these "hidden" iconsets, they are now categorized under "Deprecated / Unavailable" (which is excluded by default via user settings) and displayed with visual indicators throughout the UI.

  • Icon sets with hidden: true are now assigned to the "Deprecated / Unavailable" category instead of being filtered out at the data preparation stage
  • Visual indicators (orange information icon, dashed borders, line-through styling, reduced opacity) are added across multiple components to clearly mark deprecated icon sets
  • The "Deprecated / Unavailable" category is added to the default excluded categories list, maintaining the current user experience while allowing manual discovery

Reviewed changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
scripts/prepare.ts Removes the filter that excluded hidden collections; instead assigns them to "Deprecated / Unavailable" category
src/data/index.ts Adds hidden property to CollectionInfo interface with documentation
src/store/localstorage.ts Adds "Deprecated / Unavailable" to default excluded categories
src/pages/index.vue Adds sticky category headers, filters for available categories, and improves platform detection for keyboard shortcuts
src/components/IconSet.vue Displays deprecation indicator icon with tooltip in the icon set header
src/components/Drawer.vue Shows deprecation indicator icon next to collection names in the drawer
src/components/CollectionEntry.vue Adds comprehensive visual styling for hidden collections including dashed borders, line-through text, reduced opacity, and deprecation indicator icon
unocss.config.ts Increases border base opacity from 15 to 25 for better visibility

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

/**
* The icon set was deprecated and is no longer available
*/
hidden?: boolean
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@antfu
Copy link
Member

antfu commented Dec 3, 2025

What's the reason we still need to display them?

@zyyv
Copy link
Contributor Author

zyyv commented Dec 3, 2025

What's the reason we still need to display them?

I was using an icon collection, but I don't know why it was marked hidden: ture so I couldn't find it
It is not clear to me how iconify calculates a mark as deprecated. Even if it is marked as hidden, we can let the user decide whether to display it or not. In addition, I have defaulted hidden as ture to excludes catagories

@antfu antfu merged commit fb6710e into antfu-collective:main Dec 15, 2025
7 checks passed
@cyberalien
Copy link
Collaborator

A bit of context.

Icon sets are marked as hidden when:

  • They are duplicate (several icon sets changed prefixes per author's request)
  • They are very old and have not been maintained for 8+ years and/or bugged
  • Fluent Emoji (as shown on screenshot in this repo)

Fluent Emoji is a special case and it was forced by GitHub. That icon set has icons with massive amounts of gradients and shadows, so each icon is about 30kb. It is a big icon set, there are many icons. That resulted in JSON file reaching 100mb mark. GitHub does not allow files larger than 100kb, preventing commit from being published.

On Iconify website hidden icon sets are handled this way: hidden by default, shown when user types prefix or name in filter that matches a hidden set.

I suggest adding additional check to make sure duplicates aren't shown.

@zyyv zyyv deleted the for-hidden-collections branch December 16, 2025 08:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants